Skip to content

Major Update - Splitting out extractors#189

Closed
GeekMasher wants to merge 2 commits into
mainfrom
codeql/updates
Closed

Major Update - Splitting out extractors#189
GeekMasher wants to merge 2 commits into
mainfrom
codeql/updates

Conversation

@GeekMasher
Copy link
Copy Markdown
Contributor

No description provided.

Copilot AI review requested due to automatic review settings April 10, 2025 14:37
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR splits out extractors into separate modules and updates workspace configuration.

  • Introduces a new CodeQL pack lock file with precise dependency versions.
  • Implements separate commands for extraction, generation, and autobuild in the extractor module.
  • Adjusts Cargo.toml workspace configurations and adds configurations for downgrades and the CodeQL extractor.

Reviewed Changes

Copilot reviewed 132 out of 142 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
bicep/ql/lib/codeql-pack.lock.yml New lock file specifying CodeQL dependencies versioning
bicep/extractor/src/main.rs Implements CLI commands for extract, generate, and autobuild
bicep/extractor/src/generator.rs Adds generator support with paths for dbscheme and QLL files
bicep/extractor/src/extractor.rs Adds extractor command with file list support and tracing setup
bicep/extractor/src/autobuilder.rs Implements autobuilder command using environment configuration
bicep/extractor/Cargo.toml New Cargo.toml for the extractor package
bicep/downgrades/qlpack.yml Downgrade configuration file
bicep/codeql-extractor.yml CodeQL extractor configuration file
bicep/Cargo.toml Update workspace member configuration for the bicep directory
Cargo.toml Top-level workspace config updated to include bicep/extractor module
Files not reviewed (10)
  • .codeqlmanifest.json: Language not supported
  • .gitmodules: Language not supported
  • bicep/.codeqlmanifest.json: Language not supported
  • bicep/.gitignore: Language not supported
  • bicep/codeql-extractor.code-workspace: Language not supported
  • bicep/extractor/tree-sitter: Language not supported
  • bicep/ql/lib/bicep.dbscheme.stats: Language not supported
  • bicep/ql/lib/bicep.qll: Language not supported
  • bicep/ql/lib/codeql/IDEContextual.qll: Language not supported
  • bicep/ql/lib/codeql/Locations.qll: Language not supported

Comment thread bicep/extractor/src/extractor.rs
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 10, 2025

Dependency Review

The following issues were found:

  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 1 package(s) with unknown licenses.

View full job summary

Copy link
Copy Markdown

@github-advanced-security github-advanced-security AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CodeQL-Extractor found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

@GeekMasher GeekMasher closed this Apr 11, 2025
Comment thread bicep/Cargo.lock Fixed
Comment thread bicep/Cargo.lock Fixed
Comment thread bicep/Cargo.lock
Comment thread bicep/Cargo.lock Fixed
Comment thread bicep/Cargo.lock Fixed
Comment thread bicep/Cargo.lock Fixed
Comment thread bicep/Cargo.lock Fixed
Comment thread Cargo.lock
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants